The inspection completed: 2 new issues, 1 updated code elements

Refer to this link for build results (access rights to CI server needed):
https://ci.owncloud.org//job/pull-request-analyser-ng-simple/10815/
Test PASSed.

Does this also help to fix #7128 by any chance? :)

@jancborchardt I think for most situations yes.

@jancborchardt after reading it a bit more careful, no it does not!

Good stuff, fixes the second part of #8231 for me. 👍 Another review please @owncloud/designers?

So, the other part is slightly harder. Since we currently do not fetch any data about the share. We should probably try to solve that while we try to fix #7128 et al.

@rullzer do you want to do that in this PR or a separate one?

Another review please @owncloud/designers @schiesbn :)

@jancborchardt that fix is far less trivial, so different PR.
I do however want to add some unit tests here. But that will not be tonight :)

I do however want to add some unit tests here. But that will not be tonight :)

would be great to get this done these days - THX a lot!

@DeepDiver1975 I think this code it not unit tested at all currently. So my ini tiny tests will not improve it much. And I do not really feel like adding unit tests for the whole file.

I suggest to just go forward and get rid of all the old ajax/*.php stuff before 8.2

👍